-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#3: Added release code build #5
#3: Added release code build #5
Conversation
exasol_script_languages_container_ci_setup/lib/run_start_release_build.py
Outdated
Show resolved
Hide resolved
Get the stack resource and find the codebuild in the stack.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments
And, I don't understand the build container arrow in the flow picture
exasol_script_languages_container_ci_setup/cli/commands/start_test_release_build.py
Outdated
Show resolved
Hide resolved
exasol_script_languages_container_ci_setup/lib/github_release_creator.py
Outdated
Show resolved
Hide resolved
exasol_script_languages_container_ci_setup/lib/run_start_release_build.py
Outdated
Show resolved
Hide resolved
exasol_script_languages_container_ci_setup/lib/run_start_release_build.py
Outdated
Show resolved
Hide resolved
exasol_script_languages_container_ci_setup/templates/release_build_buildspec.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the overview picture :D
exasol_script_languages_container_ci_setup/lib/generate_buildspec_common.py
Outdated
Show resolved
Hide resolved
exasol_script_languages_container_ci_setup/lib/run_start_release_build.py
Outdated
Show resolved
Hide resolved
raise RuntimeError(f"Multiple projects match {project}. Found following matches: {matching_project}") | ||
dry_run_value = "--dry-run" | ||
|
||
gh_token = os.getenv("GITHUB_TOKEN") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wouldn't it make more sense to pass the token?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the token is a credential
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
exasol_script_languages_container_ci_setup/lib/github_release_creator.py
Outdated
Show resolved
Hide resolved
exasol_script_languages_container_ci_setup/templates/release_build_buildspec.yaml
Outdated
Show resolved
Hide resolved
exasol_script_languages_container_ci_setup/templates/buildspec_batch_entry.yaml
Show resolved
Hide resolved
2. Renamed GithubReleaseCreator => GithubDraftReleaseCreator 3. Add missing link to test-release-flow.png in user_guide.md
New commands
Other changes
The parameter
aws-profile
has been changed fromrequired
tooptional
. In GH actions the default AWS profile will be used (defined by the environment variables).